Skip to content

[pull] latest from ag-grid:latest#776

Merged
pull[bot] merged 1 commit into
code:latestfrom
ag-grid:latest
Jun 12, 2026
Merged

[pull] latest from ag-grid:latest#776
pull[bot] merged 1 commit into
code:latestfrom
ag-grid:latest

Conversation

@pull

@pull pull Bot commented Jun 12, 2026

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

* refactor: consolidate number-checking utilities into shared helpers

Add _isFiniteNumber and _toFiniteNumber to utils/number.ts and export
from main-internal.ts. Replace scattered typeof+isFinite/isNaN patterns
with the shared utilities at 9 callsites across community and enterprise.

* refactor: strip verbose JSDoc from _isFiniteNumber/_toFiniteNumber

* refactor: add blank-string coercion note to _toFiniteNumber JSDoc

* refactor: introduce _clamp utility, replacing scattered Math.max/Math.min pairs

Adds `_clamp(value, min, max)` to `utils/number` and exports it from
`main-internal`. Replaces 19 identical two-call clamp patterns across
community and enterprise with a single, readable helper.

* fix build
@pull pull Bot locked and limited conversation to collaborators Jun 12, 2026
@pull pull Bot added the ⤵️ pull label Jun 12, 2026
@pull pull Bot merged commit 7798667 into code:latest Jun 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant